Skip to content

Conversation

@amine-errahmani
Copy link
Contributor

this is to make the livenessProbe and the readinessProbe get templated directly from the values.
this fixes issue #184

@amine-errahmani amine-errahmani requested a review from a team as a code owner November 23, 2025 17:07
@manuel-rw manuel-rw added the bug Something isn't working label Nov 23, 2025
@manuel-rw
Copy link
Member

@amine-errahmani can you please resolve merge conflicts?

@amine-errahmani
Copy link
Contributor Author

conflicts resolved

@oben01
Copy link
Member

oben01 commented Nov 25, 2025

@amine-errahmani Thank u for this contribution. I just left some comments

@amine-errahmani
Copy link
Contributor Author

@oben01 i just resolved conflicts again
but i don't see any comments

@oben01
Copy link
Member

oben01 commented Dec 5, 2025

@amine-errahmani in the review tab

@oben01
Copy link
Member

oben01 commented Dec 5, 2025

#185 (review)

@amine-errahmani
Copy link
Contributor Author

dosen't seem like i have access to the review tab, clicking on the link just redirects me to the conversation tab
i tried multiple browsers, but it doesn't show

@oben01
Copy link
Member

oben01 commented Dec 5, 2025

@amine-errahmani is it better now ?!

@amine-errahmani
Copy link
Contributor Author

yup, updated

@oben01
Copy link
Member

oben01 commented Dec 5, 2025

@amine-errahmani LGTM thanks :)

@oben01
Copy link
Member

oben01 commented Dec 5, 2025

@amine-errahmani could u please regenerate the documentation before I merged it

@amine-errahmani
Copy link
Contributor Author

documentation updated

@oben01 oben01 merged commit f0dee3e into homarr-labs:dev Dec 5, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants